-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to monaco 0.45 #290
Conversation
@CGNonofr awesome PR on release day of new VSCode. 🎉 Can you please release a preview version for testing. I will take a look at everything tomorrow morning. I have one request/discussion point about the versioning scheme. What do you think about the following version scheme: So, new features, breaking changes will be |
I have the feeling someone didn't read the description (: It looks like a great idea! Do you mind creating an issue or a discussion? |
Will do... |
See: #291 |
@CGNonofr did local tests. First impression is good. Can you please produce a preview release? Thank you. |
You mean a next version? It's already done |
Sorry, overlooked that. |
I'll install this version on our codebase and see if everything works. I'll probably know by Monday! |
But don't let me block the merge. If there's anything that need changing I can contribute them after as well. |
Latest finding form tests with preview version of TypeFox/monaco-languageclient#584 |
Upgrade was pretty smooth! Everything is working as expected here. The only hurdle was updating our xterm dependencies to |
for some reasons...
to prevent crashes
abae48a
to
576b108
Compare
=> 576b108 (no idea why it wasn't an issue before nor the exact source of the issue, I've failed debugging it - there is a lot of events handlers and callbacks I was unable to follow - before I noticed that difference) |
@CGNonofr happy new year! 🎇 Thank you for finding this. With monaco-editor 0.44.0 they switched to a the new DiffEditor implementation, right.? Maybe this error is somehow related/follow-up problem (this is wild guessing, though). If you release a new next version, I can verify it with our dependent library stack and then we can merge that in hopefully today. |
Happy new year (: You're probably right 🤷 published as 1.85.0-next.2 :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good now. Did extensive testing with dependent repos/examples! LGTM
awesome! thanks for the time spent! |
published as 1.85.0-next.0
also fix #289